Skip to content

fix: Show threshold chart when creating alerts for unsaved search - #704

Merged
kodiakhq[bot] merged 2 commits into
v2from
ernest/hdx-1474-create-alert-doesnt-show-threshold-preview-for-new-searches
Mar 25, 2025
Merged

kodiakhq[bot] merged 2 commits into
v2from
ernest/hdx-1474-create-alert-doesnt-show-threshold-preview-for-new-searches

Conversation

@ernestii

@ernestii ernestii commented Mar 22, 2025 •

Copy link
Copy Markdown
Contributor

Remove dependency on savedSearch.

@changeset-bot

changeset-bot Bot commented Mar 22, 2025 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6754242

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ernestii
ernestii changed the base branch from main to v2 March 22, 2025 02:08
@ernestii ernestii changed the title ernest/hdx 1474 create alert doesnt show threshold preview for new searches fix: Show threshold chart when creating alerts for unsaved search Mar 22, 2025
@ernestii
ernestii marked this pull request as ready for review March 22, 2025 02:08

@wrn14897 wrn14897 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome fix!

@kodiakhq
kodiakhq Bot merged commit ca3d8fe into v2 Mar 25, 2025
@kodiakhq
kodiakhq Bot deleted the ernest/hdx-1474-create-alert-doesnt-show-threshold-preview-for-new-searches branch March 25, 2025 14:38
knudtty pushed a commit that referenced this pull request Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants